Skip to content

Documentation update #1573

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

LaszloLango
Copy link
Contributor

JerryScript-DCO-1.0-Signed-off-by: László Langó [email protected]

@LaszloLango LaszloLango added documentation Related to documentation enhancement An improvement labels Feb 9, 2017
**Change default heap size (512K)**

```bash
python tools/build.py --mem-heap=256K
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just --mem-heap=256, this option accepts integers.

If you would like to use more than 512K, then you must enable the 32 bit compressed pointers.

```bash
python tools/build.py --cpointer-32bit=on --mem-heap=1024K
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

--mem-heap=1024

**Use system memory allocator**

```bash
python tools/build.py --system-allocator=on --jerry-libc=off
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be good to add --compile-flag=-m32 to the command or add some hints about how to build to 32bit.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bzsolt, this is why I wrote a note. :)

python tools/build.py --cpointer-32bit=on --mem-heap=1024K
```

*Note*: Memory allocator of JerryScript will allocate the full heap, when the engine starts.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't completely correct, as our memory allocator has no responsibility to allocate the heap. The heap will be allocated statically at compile time.

if the value parameter is not a string or the buffer is
not large enough for the whole string.
Copy the characters of a string into a specified cesu-8 buffer.
The '\0' character could occur in character buffer. Returns 0,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in the

JerryScript-DCO-1.0-Signed-off-by: László Langó [email protected]
@LaszloLango
Copy link
Contributor Author

@bzsolt, @dbatyai thanks for the reviews. I've updated the PR.

Copy link
Member

@zherczeg zherczeg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@dbatyai dbatyai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LaszloLango LaszloLango merged commit 32674ff into jerryscript-project:master Feb 10, 2017
@LaszloLango LaszloLango deleted the update-documentation branch April 28, 2017 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Related to documentation enhancement An improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants